Skip to content

refactor: drop dependency of unmaintained paste#19

Merged
MrCroxx merged 1 commit intoMrCroxx:mainfrom
tisonkun:nopaste
Oct 27, 2025
Merged

refactor: drop dependency of unmaintained paste#19
MrCroxx merged 1 commit intoMrCroxx:mainfrom
tisonkun:nopaste

Conversation

@tisonkun
Copy link
Contributor

@tisonkun tisonkun commented Oct 24, 2025

@tisonkun
refactor: drop dependency of unmaintained paste
d15c0cf 
Signed-off-by: tison <wander4096@gmail.com>
@codecov
Copy link

codecov bot commented Oct 27, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 93.71429% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.50%. Comparing base (b34ddf9) to head (d15c0cf).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/atomic.rs 93.75% 11 Missing ⚠️
src/base.rs 93.67% 11 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
- Coverage   95.35%   94.50%   -0.86%     
==========================================
  Files           2        2              
  Lines         474      400      -74     
==========================================
- Hits          452      378      -74     
  Misses         22       22

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

MrCroxx
MrCroxx approved these changes Oct 27, 2025
View reviewed changes
Copy link
Owner

@MrCroxx MrCroxx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you, Tison. Let's wait for the CI to pass.

@tisonkun
Copy link
Contributor Author

tisonkun commented Oct 27, 2025

@MrCroxx CI passed. Not sure why codecov complains :D

@MrCroxx
Copy link
Owner

MrCroxx commented Oct 27, 2025

@MrCroxx CI passed. Not sure why codecov complains :D

Don't worry. I reviewed the diff. It is fine. The codecov status is not required, just a reminder. Maybe I will update the rules later.

@MrCroxx MrCroxx merged commit 3aff3c5 into MrCroxx:main Oct 27, 2025
26 of 28 checks passed
@MrCroxx
Copy link
Owner

MrCroxx commented Oct 27, 2025

Merged. Thank you again, Tison.

@tisonkun
Copy link
Contributor Author

tisonkun commented Oct 27, 2025

@MrCroxx may we cut a patch release so that downstream project can upgrade to?

@tisonkun
Copy link
Contributor Author

tisonkun commented Oct 27, 2025

Ah. I found 0.2.3 now.

@tisonkun tisonkun deleted the nopaste branch October 27, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MrCroxx MrCroxx MrCroxx approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tisonkun @MrCroxx